Remove implementation of traits that can confuse users because of lack well-defining meaning - #1007

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 55.28%. Comparing base (e00ae5c) to head (7cd0fe9).
⚠️ Report is 64 commits behind head on master.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##           master    #1007      +/-   ##
==========================================
- Coverage   57.31%   55.28%   -2.04%     
==========================================
  Files          46       51       +5     
  Lines       18197    18721     +524     
==========================================
- Hits        10429    10349      -80     
- Misses       7768     8372     +604     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@dralley, It seems that this PR was not included in v0.42.0, because you pushed release before it was merged 😕

I did merge it first. Looking at the chain of events, it looks like I just forgot to get fetch upstream before I rebased against upstream/master and created the local tag.

(which in hindsight is an error-prone workflow, should have just switched back to master branch and git pull upstream master, and done the tagging there - less chance of mistakes).

Sorry. Would you like to see a re-release as 0.42.1, or just move it to the Unreleased section?

Removing trait implementation would be a breaking change. I suggest to following course of actions:

• force-push master to match the v0.42.0 tag
• cherry-pick this PR with edits to put it Unreleased section
• release v0.43.0 with the fix

Just did the first two points.